hmm.. in the premium addon I have such code and at first glance I feel it's intentional.

I have a ExprEngine::Value class that contains a virtual isUninit() method. The default implementation returns false.

I have a derived ExprEngine::BailoutValue class that overrides that method and returns false also. I feel I do want to explicitly return false in the BailoutValue and it's more dangerous to remove it than keeping it.

But well the alternative could maybe be to remove the base class implementation so it must be overridden. If the idea is that the base class method should be empty then it could be clarified in the message.